Skip to content

Local API: Support extracting about information from YouTube's new about popup#4370

Merged
FreeTubeBot merged 2 commits intoFreeTubeApp:developmentfrom
absidue:support-about-popup
Dec 21, 2023
Merged

Local API: Support extracting about information from YouTube's new about popup#4370
FreeTubeBot merged 2 commits intoFreeTubeApp:developmentfrom
absidue:support-about-popup

Conversation

@absidue
Copy link
Member

@absidue absidue commented Nov 22, 2023

Local API: Support extracting about information from YouTube's new about popup

Pull Request Type

  • Bugfix
  • Feature Implementation

Description

This pull request adds support for extracting the about information from YouTube's new popup, I also added support for showing the number of videos that the channel has. Auto-generated game channels like the Minecraft one below, still use the old about tab.

This doesn't fix the missing information on the about tab when Invidious is selected as the backend, as that requires changes on their side, to parse and include it in their API response.

This pull request also updates YouTube.js from 7.0.0 to 8.0.0, it contains two breaking changes, but one is for the library view that only logged in users have and the other one is because YouTube removed the Channels tab on the channel page, but we have "always" extracted the related channels from the home tab, so the breaking change doesn't affect us either.

Testing

Gibi ASMR (has the new about popup)
https://www.youtube.com/channel/UCE6acMV3m35znLcf0JGNn7Q

Minecraft - Topic (has the old about tab)
https://www.youtube.com/channel/UCQvWX73GQygcwXOTSf_VDVg

Remember to run yarn install after checking out this branch, so that you get the YouTube.js update. You will also want to run yarn install again after switching away from this branch, to downgrade YouTube.js again, otherwise you'll get errors because the development branch doesn't support the new about data structure.

@absidue absidue changed the title Support extracting about information from YouTube's new about popup Local API: Support extracting about information from YouTube's new about popup Nov 22, 2023
@absidue absidue force-pushed the support-about-popup branch from d579e6c to ecf082a Compare December 2, 2023 15:36
@absidue
Copy link
Member Author

absidue commented Dec 2, 2023

Remember to run yarn install after checking out this branch, so that you get the YouTube.js update. You will also want to run yarn install again after switching away from this branch, to downgrade YouTube.js again, otherwise you'll get errors because the development branch doesn't support the new about data structure.

@absidue absidue marked this pull request as ready for review December 2, 2023 15:42
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) December 2, 2023 15:42
@github-actions github-actions bot added PR: dependencies Pull requests that update a dependency file PR: waiting for review For PRs that are complete, tested, and ready for review labels Dec 2, 2023
@absidue absidue added PR: dependencies Pull requests that update a dependency file and removed PR: dependencies Pull requests that update a dependency file labels Dec 2, 2023
Copy link
Member

@PikachuEXE PikachuEXE left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can view content in about tab for both channels

Off topic: I always use yarn && yarn dev to avoid library version issue

@FreeTubeBot FreeTubeBot merged commit 6541695 into FreeTubeApp:development Dec 21, 2023
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Dec 21, 2023
@absidue absidue deleted the support-about-popup branch December 21, 2023 08:16
absidue added a commit that referenced this pull request Feb 23, 2024
…out popup (#4370)

* Support extracting about information from YouTube's new about popup

* Update YouTube.js from 7.0.0 to 8.0.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants